feat: add scramble flag for development text scrambling #745
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit introduces a new scramble option to help reduce distractions
during development by scrambling text using rot13. When enabled via the
new `--scramble` flag, text displayed in various views is transformed,
making it easier to focus on layout and behavior without reading the
actual content.
App & Args Updates
- Added a `scramble: bool` field to the main application state (in `app.rs`).
- Extended argument parsing (in `args.rs`) to recognize the `--scramble` flag.
NoteOptions Enhancement
- Introduced a new bit flag `scramble_text` in `NoteOptions` with
corresponding setter/getter methods.
UI Adjustments
- Propagated the scramble flag through note rendering functions across
navigation, timeline, and note view modules.
- Updated several UI components (e.g., in `nav.rs`, `route.rs`, and
`contents.rs`) to accept and apply the new note options.
Rot13 Implementation
- Implemented a helper function (`rot13`) to scramble text
conditionally when the scramble option is enabled.
This feature is intended for development builds only, offering a way to
obscure text content during UI tweaks and testing.
Signed-off-by: William Casarin <[email protected]>
- Introduced a new "no_media" boolean in ColumnsArgs to capture the --no-media flag. - Updated NoteOptions to include a setting for hiding media, configured from parsed arguments. - Refactored Damus to consolidate note options (textmode, scramble, and no-media) into a single NoteOptions field. - Modified navigation UI rendering to pass the unified note_options. This change allows users to disable media display via the --no-media flag. Signed-off-by: William Casarin <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Inspired by @danieldaquino 's change in damus iOS...
This commit introduces a new scramble option to help reduce distractions during development by scrambling text using rot13. When enabled via the new
--scrambleflag, text displayed in various views is transformed, making it easier to focus on layout and behavior without reading the actual content.App & Args Updates
Added a
scramble: boolfield to the main application state (inapp.rs).Extended argument parsing (in
args.rs) to recognize the--scrambleflag.NoteOptions Enhancement
scramble_textinNoteOptionswith corresponding setter/getter methods.UI Adjustments
Propagated the scramble flag through note rendering functions across navigation, timeline, and note view modules.
Updated several UI components (e.g., in
nav.rs,route.rs, andcontents.rs) to accept and apply the new note options.Rot13 Implementation
rot13) to scramble text conditionally when the scramble option is enabled.This feature is intended for development builds only, offering a way to obscure text content during UI tweaks and testing.